Skip to content

[ci skip] Add myself as ext/session maintainer - #22976

Closed
jorgsowa wants to merge 1 commit into
php:PHP-8.4from
jorgsowa:add-jorgsowa-session-maintainer
Closed

[ci skip] Add myself as ext/session maintainer#22976
jorgsowa wants to merge 1 commit into
php:PHP-8.4from
jorgsowa:add-jorgsowa-session-maintainer

Conversation

@jorgsowa

Copy link
Copy Markdown
Contributor

I have made a number of PRs already to the extension, and I intend to improve its state even more.

@LamentXU123

Copy link
Copy Markdown
Member

The CODEOWNER file only works for someone with commit rights to the github repo.

@jorgsowa

Copy link
Copy Markdown
Contributor Author

@iluuu1994 can you tell me what the current process is? I have been contributing to the php-src for a few years already.

@NattyNarwhal

Copy link
Copy Markdown
Member

Be sure to target the oldest supported branch; per the comments GH checks the target branch's CODEOWNERS for a PR.

@jorgsowa
jorgsowa force-pushed the add-jorgsowa-session-maintainer branch from 58c142b to 9858b5e Compare August 4, 2026 22:03
@jorgsowa
jorgsowa changed the base branch from master to PHP-8.4 August 4, 2026 22:04
@TimWolla

TimWolla commented Aug 5, 2026

Copy link
Copy Markdown
Member

@jorgsowa Can you please fill in https://www.php.net/git-php.php?

@iluuu1994

Copy link
Copy Markdown
Member

@jorgsowa Your request was approved. Please familiarize yourself with https://wiki.php.net/vcs/gitworkflow. Please always seek approvals for PRs before merging. Let me know if you have any questions.

LamentXU123 added a commit that referenced this pull request Aug 6, 2026
* PHP-8.5:
  [ci skip] Add Jorg Sowa as ext/session maintainer (#22976)
@LamentXU123

Copy link
Copy Markdown
Member

Thank you and welcome!

pull Bot pushed a commit to ppker/php-src that referenced this pull request Aug 6, 2026
* PHP-8.4:
  [ci skip] Add Jorg Sowa as ext/session maintainer (php#22976)
@TimWolla

TimWolla commented Aug 6, 2026

Copy link
Copy Markdown
Member

@LamentXU123 This PR would've been a good opportunity for Jorg to test whether everything works 😉

@jorgsowa

jorgsowa commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

Thank you all for acceptance! I will check how it works with my other PRs in coming days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants